Skip to content

Replace deprecated hiltViewModel#6118

Merged
TimoPtr merged 1 commit intofeature/use_new_onboardingfrom
feature/deprecated_hiltviewmodel
Dec 3, 2025
Merged

Replace deprecated hiltViewModel#6118
TimoPtr merged 1 commit intofeature/use_new_onboardingfrom
feature/deprecated_hiltviewmodel

Conversation

@TimoPtr
Copy link
Member

@TimoPtr TimoPtr commented Dec 1, 2025

Summary

hiltViewModel extension has been moved to another package this PR use the new one.

Checklist

  • New or updated tests have been added to cover the changes following the testing guidelines.
  • The code follows the project's code style and best_practices.
  • The changes have been thoroughly tested, and edge cases have been considered.
  • Changes are backward compatible whenever feasible. Any breaking changes are documented in the changelog for users and/or in the code for developers depending on the relevance.

@jpelgrom
Copy link
Member

jpelgrom commented Dec 1, 2025

Do all these PRs need to merge into feature/use_new_onboarding? That branch is becoming quite big.

@TimoPtr
Copy link
Member Author

TimoPtr commented Dec 1, 2025

Do all these PRs need to merge into feature/use_new_onboarding? That branch is becoming quite big.

This or we merge the use_new_onboarding on main then all of these other branches.
Or we can wait until I have most of block PR's ready.

@jpelgrom
Copy link
Member

jpelgrom commented Dec 1, 2025

This or we merge the use_new_onboarding on main

That depends on your confidence :) But some of these changes, like this PR and part of the string cleanup, don't really need feature/use_new_onboarding.

@TimoPtr
Copy link
Member Author

TimoPtr commented Dec 3, 2025

Sorry I commented on the wrong PR yes it was possible to do this PR on main... Having it here is going to make my life easier. (This comment was for the strings PR)

@TimoPtr TimoPtr merged commit ccdc8af into feature/use_new_onboarding Dec 3, 2025
19 checks passed
@TimoPtr TimoPtr deleted the feature/deprecated_hiltviewmodel branch December 3, 2025 13:47
TimoPtr added a commit that referenced this pull request Dec 17, 2025
* Remove LaunchActivity

* Merge onboarding and app

* Update lint baselines

* Add battery optimization permission to onboarding (#6095)

* Request battery optimization permission when leaving Location screen

* Fix crash on Android 6 with Github icon

* Replace deprecated hiltViewModel (#6118)

* Add notification permission bottom sheet (#6094)

* Store user preference for Notification dialog

* Add notification bottom sheet on the webview activity

* Remove strings that are not used anymore (#6117)

---------

Co-authored-by: Joris Pelgröm <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants